-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Unreal Engine: Add migration guide #14186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bundle ReportChanges will decrease total bundle size by 105.9kB (-0.47%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
b1929d7 to
d6d0ef5
Compare
|
|
||
| - Replace `USentryUserFeedback` class references with `USentryFeedback` | ||
| - Replace `USentrySubsystem::CaptureUserFeedback` function usages with `USentrySubsystem::CaptureFeedback` | ||
| - Replace `USentryLibrary::CreateSentryUserFeedback` function usages with `USentryLibrary::CreateSentryFeedback` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a note on the eventId and associated id.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It already mentions that event id is now optional and only message is required to create a new feedback. Do we need anything else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a matter of how descriptive do we want to be
Often the best way to read these docs, is to see an example of what you were writing, and what's the new way of doing the same thing.
Otherwise, you're left trying to understand the new API from scratch and doing the mapping in ur head
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added feedback submission example in 4e29a73
|
|
||
| #### Global scope configuration | ||
|
|
||
| With the removal of `USentrySubsystem::ConfigureScope` global scope values (tags, context, attachments, etc.) should now be configured directly via the subsystem interface. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we have a code snippet as an example of what it used to be before and what has to be now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added example for scope configuration in 7a0a6ca
Co-authored-by: Bruno Garcia <[email protected]>
* 'master' of https://github.com/getsentry/sentry-docs: Layout updates; widening content div, centering and responsiveness, scrollbar update (#14766) Document default sdk logging attributes (#14353) Bump API schema to b8165143 (#14764) fix(OnboardingOptionButtons) Remove sticky css on mobile (#14690) Unreal Engine: Add migration guide (#14186) docs(ai): Update AI agent integrations list (#14769) fix: Order versioned docs select items by recency (#14768) ref(python): Update wording regarding async init (#14631) docs(aws-integration): Remove alert that ESM is not supported (#14756) fix(feedback): Update docs with SwiftUI notes (#14227) chore(docs): Remove explicit openai agents integration registration (#14718) Update list of states subject to tax on /pricing (#14763) ref(replay): Refactor pages inside the Product Walkthrough > Replay area (#14712) docs(replay): Move FAQ Replay for Android questions into dedicated troubleshooting page (#14709)
This PR adds Unreal Engine SDK migration guide for the upcoming
1.0.0plugin release.Related to:
USentrySettingssentry-unreal#1018Epic Account IdandLogin Idfrom crash context sentry-unreal#886beforeSendhandler during post-loading sentry-unreal#589